We are migrating the bug tracker to github Issues. This is now the preferred way to report NASM bugs.

Self-registration is disabled due to spam issue (mail gorcunov@gmail.com or hpa@zytor.com to create an account)

Bug 3392481 - nasmdoc.pdf code example contains U+2212 'MINUS SIGN' instead of U+002D 'HYPHEN-MINUS'
Summary: nasmdoc.pdf code example contains U+2212 'MINUS SIGN' instead of U+002D 'HYPH...
Status: CLOSED FIXED
Alias: None
Product: NASM
Classification: Unclassified
Component: Documentation (show other bugs)
Version: 2.13.xx
Hardware: All All
: Medium normal
Assignee: nobody
URL:
Depends on:
Blocks:
 
Reported: 2018-05-13 05:52 PDT by E. C. Masloch
Modified: 2020-07-02 21:08 PDT (History)
3 users (show)

Obtained from: Binary from nasm.us
Generated by: ---
Bug category:
Observed for: ---
Regression: ---
Regression since:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description E. C. Masloch 2018-05-13 05:52:54 PDT
I noticed that the PDF file at https://www.nasm.us/xdoc/2.13.03/nasmdoc.pdf contains, eg on page 51, the following text:

This can be done by the following definition:

%macro  multipop 1−* 
  %rep %0 
  %rotate −1 
        pop     %1 
  %endrep 
%endmacro

Note that the minus sign in the parameter count specifier and in the %rotate directive is an U+2212 'MINUS SIGN', not the ASCII-range U+002D 'HYPHEN-MINUS'. This seems likely related to the following halibut commit:

https://git.tartarus.org/?p=simon/halibut.git;a=commit;h=73e8c7d1b4ac77ec1b5acc700cb3af277a150bcf

(Halibut is a free documentation preparation system, the predecessor scripts of which are used in nasm.)

Looking at the source code for the documentation, it does use U+002D as it should: http://repo.or.cz/nasm.git/blob/ef7d18bfbb899fe8e44f4fae8d6a12e2742f69f5:/doc/nasmdoc.src#l2693

This bug seems to be the cause of https://bugzilla.nasm.us/show_bug.cgi?id=3392462
Comment 1 H. Peter Anvin 2020-07-02 21:08:53 PDT
Seems the definition of 'hyphen' and 'minus' in PostScript was rather ill-defined... swapping the two seems to help.